chore(api): bump etl for playlists.last_added_to (#348)#933
Merged
Conversation
Bumps go-openaudio to e8586ac. Primary: #348 fixes playlists.last_added_to never being written (drives "recently added to" sort + playlist-update notifications). Also carries #345/#346 (TrackCollaborator entity), which adds migration 0033 creating the new track_collaborators table. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
4 tasks
raymondjacobson
added a commit
that referenced
this pull request
Jun 9, 2026
…934) ## Summary Bumps `github.com/OpenAudio/go-openaudio` and `.../pkg/etl` from `e8586ac` (#933) to **`bda0b8b`**. **Primary:** go-openaudio **#350** — - persists `developer_apps.image_url` (previously dropped; 271 prod apps have it), gated by the legacy `is_fqdn` check; - rune-counts the comment `video_url` and oauth `redirect_uri` length caps (byte-vs-rune over-strictness, same class as the name/bio fix). **Also carried** (main advanced past the baseline): - #349 — let a collaborator leave a track after accepting (reuses the `track_collaborators` table). - #351 — accept any genre string up to 100 chars (relaxes the genre allowlist). `core-indexer` runs this vendored image, so the bump is required to ship the fixes. ## Changes - `go.mod` / `go.sum`: both go-openaudio modules → `v1.3.1-0.20260609040102-bda0b8b592f5` ## Deploy note — no migration Latest ETL migration stays **0033** (#349 reuses the existing `track_collaborators` table). Everything here is code-only — no schema change, no special rollout. ## Test plan - [x] `go mod tidy` — only go.mod/go.sum changed - [x] `go build ./...` passes - [x] resolved module contains the image_url fix (`validatedAppImageURL`); migration unchanged (0033) - [ ] Post-deploy: set a dev-app icon and a long multibyte comment video on the test account → both persist 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Bumps
github.com/OpenAudio/go-openaudioand.../pkg/etlfromc5fcacf(#924) toe8586ac.Primary: go-openaudio #348 — fixes
playlists.last_added_tonever being written (perpetually NULL), which silently broke the "recently added to" sort and the playlist-update notification. Now set to the block time of the most recent track add (code-only in #348).Also carried (main advanced past the baseline): #345/#346 — the TrackCollaborator entity. This adds ETL migration 0033, which
CREATE TABLE IF NOT EXISTS track_collaborators(a new, empty table) plus one index on it.core-indexerruns this vendored image, so the bump is required to ship the fix.Changes
go.mod/go.sum: both go-openaudio modules →v1.3.1-0.20260609013415-e8586ac9ce16Deploy note — migration 0033 is low risk
0033 creates a brand-new empty table + index with
IF NOT EXISTS. There's no existing data to scan and noACCESS EXCLUSIVElock on a populated table (unlike the 0031 slug index), so it applies instantly — no pre-build/CONCURRENT step needed. Thelast_added_tofix itself is code-only (no migration).Test plan
go mod tidy— only go.mod/go.sum changedgo build ./...passeslast_added_towrites (LastAddedToin playlist_row.go) and migration0033last_added_to, and a rename preserves it🤖 Generated with Claude Code